Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updating to master branch #1

Open
wants to merge 6,344 commits into
base: master-v1.4.2
Choose a base branch
from
Open

Updating to master branch #1

wants to merge 6,344 commits into from

Conversation

jkim485
Copy link

@jkim485 jkim485 commented Jun 30, 2017

What? Why?

Fix # .

Changes proposed in this pull request:

  • a
  • b

Checks

  • Runs on Heroku.
  • Follows 7 rules of great commit messages. For most PRs a single commit should suffice, in some cases multiple topical commits can be useful. During review it is ok to see tiny commits (e.g. Fix reviewer comments), but right before the code gets merged to master or rc branch, any such commits should be squashed since they are useless to the other developers. Definitely avoid merge commits, use rebase instead.
  • Follows the Google Style Guide.
  • Make sure your commit messages end with a Signed-off-by string (this line
    can be automatically added by git if you run the git-commit command with
    the -s option)
  • If this is a feature, the PR is to rc. If this is a bug fix, the PR is to
    hotfix.

Any screenshots or GIFs?

If this is a new visual feature please add a before/after screenshot or gif
here with e.g. GifGrabber.

Notify reviewers

Read our Pull request merging
policy. If you are part of the
cBioPortal organization, notify the approprate team (remove inappropriate):

@cBioPortal/frontend
@cBioPortal/backend
@cBioPortal/devops

If you are not part of the cBioPortal organization look at who worked on the
file before you. Please use git blame <filename> to determine that
and notify them here:

haynescd and others added 30 commits January 29, 2024 14:03
@haynescd
🔒 Fix Integration Security Tests
6bad1dd
@haynescd
Remove port binding for mysql
7ad2958
@haynescd
Remove Public group from test integration studies and disable filter …
84e599d 
…groups by appname
@haynescd
🔒 Add support for configurable Role Attribute name Property
323fca3
@haynescd
Add Todo and Add another Study for testing
7f4feb2
@haynescd
Sonar Cloud Fixes
c54b61f
@haynescd
Add dynamic host to legacy swagger api
0ff82e5
@haynescd
Remove default springdoc properties
5ee68c5
@JREastonMarks @haynescd
Fix some issues and code cleanup (#10567)
780cc50 
Adds integration testing to sonarcube analysis.
Fixes some sonarcube issues that were raised
@JREastonMarks @haynescd
Adds support for trailing slashes (#10572)
18e1224
@haynescd
🔒 Fix Custom OAuth Config to grab Authorities
1758c27
@haynescd
🔒 Add Cors Config Back
8e151bc
@haynescd
Remove auto-gen from intellij
2595da2
@haynescd
Fix SAML Security Integration tests by allowing cors
4bccc62
@haynescd
Merge pull request #10579 from cBioPortal/demo-6.0.0
3cfca1e 
Pulls fixes made in Demo 6.0.0
@qlu-cls
add custom driver related properties
4873c7b
@qlu-cls
Changes snpOnly from Boolean to boolean
f6fbacc
@qlu-cls
Merge remote-tracking branch 'upstream/master' into mutation-data-counts
5b82f6d
@uklineale
add structural variant count back to study mapper (#10582)
6a0912a
@qlu-cls
add more comments in the code
fa710fb
@qlu-cls
remove import * from all changes in this PR
328dc93
@7xuanlu
Merge branch 'master' into mutation-data-counts
f40e1f3
@inodb
fix rfc list link
24c65b9
@haynescd
🔒 Add 401 response when not authorized exception is thrown instead of…
67dbdcb 
… redirect
@haynescd
Merge pull request #10590 from cBioPortal/feature/Add-Api-401-Back
5dcd3b7 
🔒 Add 401 response when not authorized exception is thrown instead of redirect
@haynescd
🔒 Add Saml Plus Basic Auth
7e8eb16
@haynescd
Remove debug
db00084
@haynescd
Add Bcrypt Encoder for basic.password storage
f549ec3
@haynescd
Remove Deprecated Filter
ebfabb2
@haynescd
Merge pull request #10588 from cBioPortal/feature/Add-SAML-Basic-Auth
f0d06e1 
🔒 Add Saml Plus Basic Auth
hweej and others added 30 commits October 7, 2024 17:22
@hweej
Add resource-data-all endpoint and MyBatis SQL methods
728951c
@zeynepkaragoz
updated the hyve team and alumni in about-us (#11038)
66bc067
@sbabyanusha
Merge branch 'master' into sbabyanusha-patch-3
9751592
@dippindots
Merge pull request #11042 from cBioPortal/sbabyanusha-patch-3
9862072 
Update News.md
@sbabyanusha
Update News-Genie.md
46a4565
@sbabyanusha
Merge pull request #11088 from cBioPortal/sbabyanusha-patch-4
daa5bf4 
Update News-Genie.md
@hweej
Add cacheable method for fetching all data resources by study
b50a31f
add name to team
1140812
@inodb
Merge pull request #11121 from zainasir/docs
0a86307 
add name to team
@tmazor
Update OS/PFS FAQ
1e844bf
@inodb
Merge pull request #11123 from cBioPortal/tmazor-patch-2
2284f83 
Update OS/PFS FAQ
@hweej
Resource definition must be specific to the cancer_study context
78541fe
@inodb
Merge pull request #11051 from hweej/11004/resource-data-expand-all-r…
8e1a02b 
…esults

Add resource-data-all endpoint and MyBatis SQL methods
@zainasir
Update pom.xml
6d589ad
@zainasir
Prepare for v6.0.19
8c934c4
@inodb
Fix cache annotation resource data
79e72ab
@haynescd
🔒 Update Default Filter Chain Ordering for Spring Security
8e4d61a
@inodb
Merge pull request #11152 from cBioPortal/fix-cache-annotation-resour…
df9d9a5 
…ce-data

Fix cache annotation resource data
@dippindots
Merge pull request #11153 from cBioPortal/demo-fix-spring-security-fi…
f5574ee 
…lter-chain-ordering

🔒 Update Default Filter Chain Ordering for Spring Security
@zainasir
Frontend v6.0.19
93915d4
@zainasir
Prepare for v6.0.20
a925541
@inodb
Add Chromoscope News (#11133)
d02df6c 
* Add Chromoscope News
@zainasir
Frontend v6.0.20
31073bb
@zainasir
Prepare for v6.0.21
a431877
@forus
Rfc85 dynamic virtual study (#11040)
38f5060 
* Add isDynamic field to Virtual Study in Session Service

* Query sample IDs using Virtual Study View Filters is it's of dynamic type

* Rename boolean varialbe name isDynamic -> dynamic

* Refactor dynamic virtual study population

Extract smaller functions

* Fix code smells discovered by sonarlint

- Use the constructor dep. injection
- return value withou assigning it to a variable

* Add web mvc test for fetching static and dynamic virtual studies
@inodb
Add ICI note to chromoscope news (#11164)
a4e8f8b 
* Add ICI note to chromoscope news
@leexgh @inodb
Add Mutation Assessor v4 news (#11148)
c6d4050 
---------

Co-authored-by: Ino de Bruijn <[email protected]>
@nbentoneves
Remove unused noauthsessionservice property (#11169)
d2bb276
@leexgh
Add AlphaMissense news (#11202)
0c37b09
@inodb
fix indentation in news
4d77797
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.